Skip to content

Adjust Github Actions to use Helm deployment - #2

Merged
stplasim merged 12 commits into
mainfrom
ab-342-adjust-github-actions-to-use-helm-deployment
Jan 23, 2025
Merged

Adjust Github Actions to use Helm deployment#2
stplasim merged 12 commits into
mainfrom
ab-342-adjust-github-actions-to-use-helm-deployment

Conversation

@stplasim

Copy link
Copy Markdown
Contributor

No description provided.

@stplasim
stplasim requested review from ThoSap and alexlanz January 23, 2025 10:11
Comment thread readme.md Outdated
#### Example

```yaml
- uses: ./.github/actions/helm-deploy

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- uses: ./.github/actions/helm-deploy
- uses: aboutbits/github-actions-kubernetes/helm-deploy@v1

Comment thread readme.md

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What do you think about renaming aboutbits/github-actions-kubernetes/deploy to aboutbits/github-actions-kubernetes/kubectl-deploy? We could just simply release everything with version 2 then it should not be a problem and we have it more evident for the future?

Comment thread readme.md Outdated
Comment on lines +80 to +81
| `working-directory` | `.` | The working directory |
| `chart-path` | `.` | Path to the Helm chart |

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here you are referencing twice a folder, but one time you call it directory and one time path. Would it make sense to align the names?

@stplasim
stplasim requested a review from alexlanz January 23, 2025 13:11
Comment thread helm-deploy/action.yml Outdated
Comment on lines +8 to +9
chart-path:
description: 'Path to the Helm chart'

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
chart-path:
description: 'Path to the Helm chart'
chart-directory:
description: 'The directory of the Helm chart'

Comment thread readme.md

```yaml
- uses: aboutbits/github-actions-kubernetes/deploy@v1
- uses: aboutbits/github-actions-kubernetes/kubectl-deploy@v2

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please adjust also the title: Deploy to Kuberentes using kubectl.

@alexlanz alexlanz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

`

git tag v1
git push --tags

Replacing an already existing minor release:

git tag -d v1
git push origin :refs/tags/v1
git tag v1
git push --tags

`

Comment thread readme.md

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When I increace the verison, then I adjust also always in the readme under the section Versioning all the references of v1 to v2.

@stplasim
stplasim merged commit e6325e2 into main Jan 23, 2025
@stplasim
stplasim deleted the ab-342-adjust-github-actions-to-use-helm-deployment branch January 23, 2025 13:28

@ThoSap ThoSap left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very nice 😎

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants